Engineering team fixes - #1596
Conversation
d215steinberg
commented
Sep 3, 2026
d215steinberg
commented
- Beefed up agent and task definitions to prevent backend_agent from writing front-end code and unit tests.
- Explicitly specify UTF-8 encoding (not the default for Windows - Green check and red x were not showing up
- Append stderr to test output and specify that output consists of just the test summary (previously, test_summary.md contained the test code.
- Exclude Docker's .venv from the bind-mounted sandbox to prevent Windows-hostile Linux symlinks from seeping in.
- Remove Sandbox directory on error for Windows to prevent starting up with corrupt sandbox
- Specify no-sync on uv add gradio when resetting Sandbox to prevent Windows lockout.
|
Hey, The most common issue is that you've included a file outside community-contributions. This would impact all students on the course. You can see this by going to the "Files changed" tab in this PR; the only changes should be to files in community-contributions. If there's a good reason to change files outside community-contributions, like an important bug fix, then please let me know (and thank you!) The 2nd most common reason is that you've not cleared outputs of Notebooks, leading to large code changes. Instructions at the link above. The 3rd common reason is that you've included too many changes, such as many thousands of lines of code, or a large number of files. I'm grateful for your work, but see the link above for ways we can include this without the repo getting too large. The PR details in this Github screen shows you the files changed, and at the top right the number of lines added (in Green) and deleted (in Red) by this PR. Typically the Green number should be less than 2,000 and the red number should be 0. I very much look forward to merging when this is addressed. Thanks so much |
2 participants
